Skip to content

Conversation

@Ricardo-MC
Copy link

This commit adds a smart button in the payment order form view to access the related payments easily. It also provides an action to display those payments and a counter field.

This improvement facilitates navigation and batch payment creation directly from the payment order.

@Ricardo-MC Ricardo-MC force-pushed the 18-imp-account_payment_order branch from f4431fe to 9bede66 Compare September 23, 2025 14:33
Copy link
Contributor

@celm1990 celm1990 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Ricardo-MC Ricardo-MC force-pushed the 18-imp-account_payment_order branch from 9bede66 to 83022b1 Compare October 24, 2025 14:49
This commit adds a smart button in the payment order form view to access
the related payments easily. It also provides an action to display those
payments and a counter field.

This improvement facilitates navigation and batch payment creation
directly from the payment order.
@Ricardo-MC Ricardo-MC force-pushed the 18-imp-account_payment_order branch from 83022b1 to 349aeb6 Compare October 24, 2025 14:58
@Ricardo-MC
Copy link
Author

@celm1990,
Thank you very much for the correction.
Changes above.

Copy link
Contributor

@celm1990 celm1990 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pedrobaeza could you please review this?

@pedrobaeza pedrobaeza added this to the 18.0 milestone Oct 27, 2025
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be only shown when the order is confirmed. Isn't this already present in previous versions? If so, you should cherry-pick the commit instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants